Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve tests, correctly trigger warnings, and fix output markdown format #32

Merged
merged 9 commits into from
Oct 24, 2024

Conversation

tboby
Copy link
Contributor

@tboby tboby commented Oct 24, 2024

WHAT

WHY

Tests are good.

This makes it easier for next steps: probably actually addressing the low-hanging fruit issues, binlog parsing tests, and formatting.

src/Library.fs Show resolved Hide resolved
@baronfel
Copy link
Collaborator

What's the WIP pipeline for?

@tboby
Copy link
Contributor Author

tboby commented Oct 24, 2024

What's the WIP pipeline for?

It's something fiddly: it's the source you want to set up the nuget.config for your new test folder, but it seemed that if you have it in the pipeline then it can't restore from nuget.org.

See https://github.com/ionide/KeepAChangelog/actions/runs/11494434344/job/31991923809#step:6:206

I thought it was going to be a series of pipeline fixes that I'd squash after but then it worked with one edit...

@baronfel
Copy link
Collaborator

Ok, so this is good and you have no more WIP for it?

@tboby
Copy link
Contributor Author

tboby commented Oct 24, 2024

Ok, so this is good and you have no more WIP for it?

Yup!

@baronfel baronfel merged commit f6dd104 into ionide:main Oct 24, 2024
1 check passed
@tboby tboby deleted the improve-tests branch October 24, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants